Skip to content

Claude/improve template icons z93n5#2

Merged
ypriverol merged 2 commits intomainfrom
claude/improve-template-icons-Z93n5
Apr 30, 2026
Merged

Claude/improve template icons z93n5#2
ypriverol merged 2 commits intomainfrom
claude/improve-template-icons-Z93n5

Conversation

@ypriverol
Copy link
Copy Markdown
Contributor

@ypriverol ypriverol commented Apr 30, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Style
    • Replaced text-based icon glyphs with improved monochrome icons for enhanced visual clarity and consistency across the interface.
    • Refined alignment, sizing, and styling of icons in category headers, template selection checkboxes, dev-toggle button, and selected template chips for a more polished appearance.

claude added 2 commits April 30, 2026 14:39
@claude
feat: replace emoji template icons with inline SVG set
d3c7b2a 
Swaps the emoji-based TEMPLATE_ICONS for a curated, monochrome SVG icon
library (Lucide-style, inlined — no network dependency). Picks more
professional glyphs for each template: target for oncology, fish/bug/leaf
for vertebrates/invertebrates/plants, intestine coil for human-gut, etc.
Icons inherit currentColor so they tint correctly in cards, headers, and
selected chips.
@claude
refine: clearer dna/petri/erlenmeyer template icons
6a22340 
Swaps three SVGs that read poorly at 14-16px:
- dna: stacked waves -> ladder (rails + rungs)
- microscope -> petri dish (matches cell-lines semantically)
- flask-round -> erlenmeyer (cleaner silhouette)
@qodo-code-review
Copy link
Copy Markdown

qodo-code-review Bot commented Apr 30, 2026 

ⓘ You've reached your Qodo monthly free-tier limit. Reviews pause until next month — upgrade your plan to continue now, or link your paid account if you already have one.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6c392feb-d197-4794-aa7c-b3fdbba512d3

📥 Commits

Reviewing files that changed from the base of the PR and between bca2bc5 and 6a22340.

📒 Files selected for processing (1)
  • static/index.html
📝 Walkthrough

Walkthrough

Icon rendering in the template UI has been refactored from emoji and text glyphs to inline monochrome SVGs. An SVG icon registry and resolver function were added, and multiple UI elements (category headers, checkboxes, buttons, chips) now inject SVG markup dynamically with CSS adjustments for alignment and sizing.

Changes

Cohort / File(s) Summary
SVG Icon System
static/index.html		 Added inline SVG icon registry (SVG_ICONS) and iconSVG() resolver function with fallback. Updated category headers, template checkbox icons, dev-toggle button labels, and selected template chips to use SVG icons via innerHTML. Adjusted CSS for inline-flex alignment, explicit SVG sizing, checked-state coloring, and specific positioning for toggle/dev icons.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A palette of vectors so crisp and so clean,
No emoji clutter cluttering the scene,
SVGs dance in monochrome grace,
Aligned and sized in perfect place,
With one little function to make icons sing—
Hoppy coding for everything! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/improve-template-icons-Z93n5

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

@ypriverol ypriverol merged commit 0b72e3e into main Apr 30, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ypriverol @claude